fix: remove an accidentally committed .claude gitlink - #41
Merged
Conversation
My cross-repo context bump (#39) used a script that does 'git add -A', which swept the untracked .claude/worktrees/generic-500 directory into the commit. It went in as a gitlink - mode 160000 - because that directory contains its own .git file. A gitlink with no matching .gitmodules entry breaks recursive checkout: fatal: No url found for submodule path 'services/auth-api/.claude/worktrees/generic-500' Once the workspace pointer advanced to that commit, the estate's governance-audit workflow started failing at checkout, because it clones the workspace with --recurse-submodules. Removes the gitlink and gitignores .claude/, which is local Claude Code session state and should never have been committable here. Note agent-kit deliberately commits .claude/ - it ships agents, commands, hooks and skills - so this belongs in each repo's .gitignore rather than the shared template. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is breaking the estate governance audit. My fault, and the fix is small.
What happened
My cross-repo context bump (#39) used
workspace/scripts/open-pr-across.sh, which runsgit add -A. That swept the untracked.claude/worktrees/generic-500directory into the commit — and because that directory has its own.gitfile, it went in as a gitlink (mode160000), not as files:A gitlink with no matching
.gitmodulesentry breaks recursive checkout. Once the workspace pointer advanced to that commit (workspace#62),governance-audit.ymlbegan failing:Fix
Remove the gitlink, and gitignore
.claude/so it cannot recur here.Why not the shared template
agent-kitdeliberately commits.claude/— it shipsagents/,commands/,hooks/,settings.jsonandskills/. So this belongs in individual repos'.gitignore, not inrepo-template's.Only this repo was affected: I checked every member repo, and
auth-apiis the only one where.claude/was committed unintentionally.The wider lesson
open-pr-across.shdoinggit add -Ameans any untracked file present in a working tree gets committed by a cross-repo run. That is a sharp edge on a tool designed to touch every repo at once, and I have reported it against the script.🤖 Generated with Claude Code